Skip to content

Bump py-opendisplay to 7.2.3#171775

Merged
balloob merged 3 commits into
home-assistant:devfrom
g4bri3lDev:bump/py-opendisplay-7.2.1
May 22, 2026
Merged

Bump py-opendisplay to 7.2.3#171775
balloob merged 3 commits into
home-assistant:devfrom
g4bri3lDev:bump/py-opendisplay-7.2.1

Conversation

@g4bri3lDev
Copy link
Copy Markdown
Contributor

@g4bri3lDev g4bri3lDev commented May 21, 2026

Proposed change

Re-bump py-opendisplay from 5.9.0 to 7.2.3. The previous bump to 7.0.0 (#171088) was reverted in #171477 due to Alpine/musl build failures caused by two upstream issues, both now fixed:

  1. pyrefly (a Rust-compiled type checker) was accidentally included as a runtime dependency by PyCharm, pulling in a Rust toolchain requirement on source builds.
  2. epaper-dithering (transitive dep) had no musllinux_1_2_x86_64 wheel on PyPI, forcing Alpine builders to compile Rust from source.

Changelog
Diff

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:
  • Link to developer documentation pull request:
  • Link to frontend pull request:

Checklist

  • I understand the code I am submitting and can explain how it works.
  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist
  • I have followed the perfect PR recommendations
  • The code has been formatted using Ruff (ruff format homeassistant tests)
  • Tests have been added to verify that the new code works.
  • Any generated code has been carefully reviewed for correctness and compliance with project standards.

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly.
    Updated and included derived files by running: python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt.
    Updated by running python3 -m script.gen_requirements_all.
  • For the updated dependencies a diff between library versions and ideally a link to the changelog/release notes is added to the PR description.

To help with the load of incoming pull requests:

Copilot AI review requested due to automatic review settings May 21, 2026 20:36
@home-assistant home-assistant Bot added cla-signed dependency Pull requests marked as a dependency upgrade integration: opendisplay small-pr PRs with less than 30 lines. by-code-owner Quality Scale: silver labels May 21, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the OpenDisplay integration to use a newer py-opendisplay version and adapts the integration code to the library’s updated API.

Changes:

  • Bump py-opendisplay from 5.9.0 to 7.2.1 across dependency definitions.
  • Update the OpenDisplay image upload call to use the new tone keyword argument.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
requirements_all.txt Updates the pinned py-opendisplay version used in the aggregated requirements list.
homeassistant/components/opendisplay/services.py Adjusts the integration to match the updated py-opendisplay API parameter name.
homeassistant/components/opendisplay/manifest.json Updates the integration’s runtime dependency pin to py-opendisplay==7.2.1.

Comment thread homeassistant/components/opendisplay/services.py
@g4bri3lDev g4bri3lDev marked this pull request as ready for review May 21, 2026 20:40
@g4bri3lDev g4bri3lDev marked this pull request as draft May 21, 2026 20:48
@g4bri3lDev g4bri3lDev changed the title Bump py-opendisplay to 7.2.1 Bump py-opendisplay to 7.2.3 May 21, 2026
Copilot AI review requested due to automatic review settings May 21, 2026 21:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread homeassistant/components/opendisplay/services.py
@g4bri3lDev g4bri3lDev marked this pull request as ready for review May 21, 2026 21:52
@balloob balloob merged commit c7754a6 into home-assistant:dev May 22, 2026
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

by-code-owner cla-signed dependency Pull requests marked as a dependency upgrade integration: opendisplay Quality Scale: silver small-pr PRs with less than 30 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants